Codecov Report

❗ No coverage uploaded for pull request base (master@fa09b2a). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #63   +/-   ##
=========================================
  Coverage          ?   99.69%           
=========================================
  Files             ?       10           
  Lines             ?     1302           
  Branches          ?        0           
=========================================
  Hits              ?     1298           
  Misses            ?        4           
  Partials          ?        0           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update fa09b2a...f1d4626. Read the comment docs.

Python seems to be missing...hm.

That's a really weird issue... maybe let's just merge it and hope it goes away soon? I have no idea what could cause that.

I think the problem with this integration is indeed that it takes far too long (i.e. while ManifoldsBase itself takes 5-8 minutes this would run 55+ minutes), I would prefer to maybe have a miniminimal test over on Manifolds for integration testing? I will close this one for now until we have a good idea.

Yes, having a reduced set of tests would make sense but we can leave it for later.